Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added preview and migration API endpoints for route migration from regex into jinja2 #1715

Merged
merged 15 commits into from
Apr 18, 2023

Conversation

iskhakov
Copy link
Contributor

@iskhakov iskhakov commented Apr 11, 2023

What this PR does

This PR adds new API endpoints for migrating routes from regex format to jinja2 format. The changes include the following:

  • filtering_term_as_jinja2 field to GET channels_filters endpoint
  • A POST endpoint channel_filters/ABCDEF123/convert_from_regex_to_jinja2/ for migrating routes to jinja2 format.

These new endpoints will allow users to preview and migrate their existing regex routes to the more flexible and maintainable jinja2 format.

Check the screenshot where this endpoints will be used
Screenshot 2023-04-14 at 09 50 23

Which issue(s) this PR fixes

Checklist

  • Unit, integration, and e2e (if applicable) tests updated
  • Documentation added (or pr:no public docs PR label added if not required)
  • CHANGELOG.md updated (or pr:no changelog PR label added if not required)

@iskhakov iskhakov requested a review from a team April 11, 2023 03:11
@iskhakov
Copy link
Contributor Author

Reverted "adding new field and data migration" as might be incompatible change for our api and terraform. I decided to make migrations manual

@iskhakov iskhakov force-pushed the iskhakov/migrate-regex-routes-to-jinja2 branch from dba067a to f9473f2 Compare April 11, 2023 05:38
@iskhakov iskhakov changed the title Iskhakov/migrate regex routes to jinja2 Migrate regex routes to jinja2 format Apr 11, 2023
@iskhakov iskhakov self-assigned this Apr 12, 2023
@iskhakov iskhakov changed the title Migrate regex routes to jinja2 format Added preview and migration API endpoints for route migration from regex into jinja2 Apr 14, 2023
@iskhakov iskhakov changed the title Added preview and migration API endpoints for route migration from regex into jinja2 [Ready] Added preview and migration API endpoints for route migration from regex into jinja2 Apr 14, 2023
@iskhakov iskhakov added pr:no changelog pr:no public docs Added to a PR that does not require public documentation updates labels Apr 14, 2023
@iskhakov iskhakov enabled auto-merge April 18, 2023 02:49
@iskhakov iskhakov changed the title [Ready] Added preview and migration API endpoints for route migration from regex into jinja2 Added preview and migration API endpoints for route migration from regex into jinja2 Apr 18, 2023
@iskhakov iskhakov added this pull request to the merge queue Apr 18, 2023
Merged via the queue into dev with commit 9d19493 Apr 18, 2023
@iskhakov iskhakov deleted the iskhakov/migrate-regex-routes-to-jinja2 branch April 18, 2023 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:no public docs Added to a PR that does not require public documentation updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants